Skip to content

TESTS: Shutdown ThreadPool after TestNodes#37123

Merged
original-brownbear merged 1 commit intoelastic:masterfrom
original-brownbear:fix-test
Jan 3, 2019
Merged

TESTS: Shutdown ThreadPool after TestNodes#37123
original-brownbear merged 1 commit intoelastic:masterfrom
original-brownbear:fix-test

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

* If the threadpool gets shut down before the testnodes we run into an error => fixed by moving to single `After` method
* Relates #36976
@original-brownbear original-brownbear added >test Issues or PRs that are addressing/adding tests :Core/Infra/Core Core issues without another label v7.0.0 v6.7.0 labels Jan 3, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

Copy link
Copy Markdown
Member

@danielmitterdorfer danielmitterdorfer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@original-brownbear
Copy link
Copy Markdown
Contributor Author

@danielmitterdorfer thanks!

@original-brownbear original-brownbear merged commit 7686ee7 into elastic:master Jan 3, 2019
@original-brownbear original-brownbear deleted the fix-test branch January 3, 2019 21:35
original-brownbear added a commit that referenced this pull request Jan 4, 2019
* If the threadpool gets shut down before the testnodes we run into an error => fixed by moving to single `After` method
* Relates #36976
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >test Issues or PRs that are addressing/adding tests v6.7.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants